Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Design fixes for recommended apps #18422

Merged
merged 1 commit into from
Dec 17, 2019
Merged

Design fixes for recommended apps #18422

merged 1 commit into from
Dec 17, 2019

Conversation

jancborchardt
Copy link
Member

Before

recommended apps before

After

recommended apps after

Fix #18363, please review @ChristophWurst @rullzer @juliushaertl

@jancborchardt jancborchardt added bug design Design, UI, UX, etc. 3. to review Waiting for reviews labels Dec 16, 2019
@jancborchardt jancborchardt added this to the Nextcloud 18 milestone Dec 16, 2019
Copy link
Member

@ChristophWurst ChristophWurst left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Screenshots looks good

Bundle was not updated

@ChristophWurst
Copy link
Member

/compile amend /

@rullzer rullzer force-pushed the design/recommended-apps branch from f70755e to 2ebfdcf Compare December 16, 2019 14:06
@rullzer
Copy link
Member

rullzer commented Dec 16, 2019

/compile amend /

Signed-off-by: Jan-Christoph Borchardt <hey@jancborchardt.net>
Signed-off-by: npmbuildbot[bot] <npmbuildbot[bot]@users.noreply.github.com>
@jancborchardt
Copy link
Member Author

@rullzer from what I can tell, the Drone failures are unrelated?

@jancborchardt
Copy link
Member Author

Also thanks to @gary-kim for the in-person review! :) Feel free to check the code if you like

@rullzer rullzer mentioned this pull request Dec 17, 2019
34 tasks
@ChristophWurst ChristophWurst added 4. to release Ready to be released and/or waiting for tests to finish and removed 3. to review Waiting for reviews labels Dec 17, 2019
@rullzer rullzer merged commit 53f94b8 into master Dec 17, 2019
@rullzer rullzer deleted the design/recommended-apps branch December 17, 2019 11:02
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
4. to release Ready to be released and/or waiting for tests to finish bug design Design, UI, UX, etc.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Dedicated app install page polishing
3 participants